Skip to content

Merges #41#42

Merged
ashfurrow merged 5 commits intomasterfrom
twittemb-feature/disposeBagable
Aug 31, 2017
Merged

Merges #41#42
ashfurrow merged 5 commits intomasterfrom
twittemb-feature/disposeBagable

Conversation

@ashfurrow
Copy link
Copy Markdown
Member

@ashfurrow ashfurrow commented Aug 31, 2017

Merges #41, fixes the CI issue. Please see the individual commits cb1368f, fd85abf, and b8a1d88
for specifics. In summary:

  • Moves tests added in disposeBag: add a HasDisposeBag protocol #41 to Demo unit test folder.
  • Moves source files back to the root of the repo.
  • Undid some changes to the Carthage Xcode project.
  • Added a note in the readme to ward off this issue in the future.

Thibault Wittemberg and others added 5 commits August 31, 2017 09:07
In this commit, we try to conform to protocol oriented programming. It is more flexible to
introduce a HasDisposeBag protocol so that every object that respect this one
can have a DisposeBag (NSObject does respect that protocol now)
@ashfurrow ashfurrow merged commit 2ffa937 into master Aug 31, 2017
@ashfurrow ashfurrow deleted the twittemb-feature/disposeBagable branch August 31, 2017 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant